Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added logarithmic smooth colorizing #253

Merged

Conversation

iaoesch
Copy link

@iaoesch iaoesch commented Jul 14, 2024

Added a new smooth colorizing method, using logarithmic value, gives smoother color gradient in chaotic places.
(New Menupoint 'Fractal/outcoloring mode/smooth log'

Also replaced outcoloer mode constants with enum s

Tested on osx with Qt7.1

@kovzol
Copy link
Contributor

kovzol commented Jul 22, 2024

Nice feature, thank you! Maybe you want to explain it by extending the tutorial text for outcoloring modes, see https://github.com/xaos-project/XaoS/wiki#out-coloring-modes for more information.

@iaoesch
Copy link
Author

iaoesch commented Jul 27, 2024

Additionally, the Table with the outcolor modes in the Wiki at
https://github.com/xaos-project/XaoS/wiki/Reference#outside-coloring-mode
needs to be updated: Adding Smooth log color and updating the number for true-color.
(That seems not to be possible by mergerequest...)

@kovzol
Copy link
Contributor

kovzol commented Jul 29, 2024

Thanks! I'll check it out soon and give you feedback.

@kovzol kovzol merged commit a191d66 into xaos-project:master Jul 30, 2024
@kovzol
Copy link
Contributor

kovzol commented Jul 30, 2024

I made some minor improvements: 7766212 (I lowercased newton and magnet, they look more uniform this way).

Please consider giving your full name and an email address to see full credits on your work.

Also, please indicate what exactly we should add in the wiki to describe the new feature. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants